Skip to content

integrated code #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
regmismriti wants to merge 1 commit into
base: main
Choose a base branch
from
Open

integrated code #7

regmismriti wants to merge 1 commit into from

Conversation

@regmismriti
Copy link

@regmismriti regmismriti commented Jun 20, 2025

📦 Pull Request Template

Thank you for your contribution! Please complete the checklist and provide relevant details below to help us review your PR effectively.

📋 Summary

This PR integrates code components from the link-steal-paper repository into the PyGIP project. The additions include modules for inductive graph splitting, model definitions, attack simulations, and training pipelines.

🧪 Related Issues

✅ Checklist

  • My code follows the project's coding style
  • I have tested the changes and verified that they work
  • I have added necessary documentation (if applicable)
  • I have linked related issues above (if any)
  • The PR is made from a feature branch, not main

🧠 Additional Context (Optional)

This integration adds functionality for link-stealing attacks in inductive graph settings. The code has been tested on graph datasets.

@Edlison
Copy link
Collaborator

Edlison commented Jul 2, 2025

Thank you for your contribution!

  1. In data/load_data.py, please avoid importing the external package graphgallery. We aim to standardize our data loading process using only torch_geometric throughout the project to reduce external dependencies.
  2. To ensure better integration with our framework, all attack methods should inherit from the BaseAttack class. This design is described in the README section on implementation. Please refactor your implementation accordingly.

Looking forward to your update! Let us know if you have any questions.

@Edlison Edlison added the need revise Need further revise label Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

need revise Need further revise

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@regmismriti @Edlison